Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Invert Docker Image publishing order #7877

Merged
merged 1 commit into from
May 18, 2022
Merged

Invert Docker Image publishing order #7877

merged 1 commit into from
May 18, 2022

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented May 18, 2022

To appear on Docker Hub (https://calendly.com/ultralytics/meet) in top down order:

  • latest
  • latest-cpu
  • latest-arm64

🛠️ PR Summary

Made with ❤️ by Ultralytics Actions

🌟 Summary

Enhancements to the Docker image publishing workflow for YOLOv5.

📊 Key Changes

  • Reformatted the branches line in the workflow for clarity.
  • Reorganized steps to build and push Docker images, specifically for the ARM64 architecture.

🎯 Purpose & Impact

  • Improved readability in the CI configuration helps maintainability.
  • By moving the ARM64 Docker image build to an earlier step, the workflow is potentially streamlined, potentially leading to faster builds.
  • Users of ARM64 architecture will receive Docker images faster and could experience improved reliability in image delivery.

To appear on Docker Hub in top down order:

- latest
- latest-cpu
- latest-arm64
@glenn-jocher glenn-jocher self-assigned this May 18, 2022
@glenn-jocher glenn-jocher merged commit 7d46c69 into master May 18, 2022
@glenn-jocher glenn-jocher deleted the update/docker branch May 18, 2022 15:23
tdhooghe pushed a commit to tdhooghe/yolov5 that referenced this pull request Jun 10, 2022
To appear on Docker Hub in top down order:

- latest
- latest-cpu
- latest-arm64
ctjanuhowski pushed a commit to ctjanuhowski/yolov5 that referenced this pull request Sep 8, 2022
To appear on Docker Hub in top down order:

- latest
- latest-cpu
- latest-arm64
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant